Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link to new (CMake) installer by default #957

Merged
merged 1 commit into from
Mar 4, 2024
Merged

Link to new (CMake) installer by default #957

merged 1 commit into from
Mar 4, 2024

Conversation

TMRh20
Copy link
Member

@TMRh20 TMRh20 commented Mar 4, 2024

  • Point to new CMake installer by default
  • Mark old installer deprecated

I think this change will make things easier for users, since the CMake installer works on every RPi version.
I've marked the old installer 'Deprecated' as well

- Point to new CMake installer by default
- Mark old installer deprecated
@TMRh20 TMRh20 merged commit 9eca153 into master Mar 4, 2024
6 checks passed
@TMRh20 TMRh20 deleted the Docs-Installer branch March 4, 2024 08:47
@2bndy5
Copy link
Member

2bndy5 commented Mar 4, 2024

It would be nice if we could sunset the old ./configure + makefiles approach. Its not easy to maintain both. If we were using just CMake, there wouldn't be a need to copy an includes.h; we could just have CMake generate it from a template includes.h.in file.

@TMRh20
Copy link
Member Author

TMRh20 commented Mar 4, 2024

I guess we could do that right now, but lets maybe leave it in place for a while (few months?) to let people adjust to the new cmake approach. I still have instructionals and videos that show the old installer, so I will likely have to leave it in place and copy over it with your CMake script.

@2bndy5
Copy link
Member

2bndy5 commented Mar 4, 2024

I was thinking in terms of years. Ultimately, the choice will be decided on how quickly people adopt the 64-bit RPi OS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants